Skip to content

Initialize line_protocol with String.new#149

Merged
bednar merged 2 commits intoinfluxdata:masterfrom
tiramizoo:fix/point-to-line-protocol-frozen-error
Mar 2, 2026
Merged

Initialize line_protocol with String.new#149
bednar merged 2 commits intoinfluxdata:masterfrom
tiramizoo:fix/point-to-line-protocol-frozen-error

Conversation

@defkode
Copy link
Contributor

@defkode defkode commented Jan 27, 2026

Closes #148

Proposed Changes

Point#to_line_protocol now works with RUBYOPT="--enable-frozen-string-literal"

Checklist

  • rake test completes successfully

Copy link
Contributor

@bednar bednar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@defkode Thanks for your PR 👍

Can you please update the CHANGELOG.md?

@defkode defkode requested a review from bednar January 30, 2026 07:25
@defkode
Copy link
Contributor Author

defkode commented Mar 2, 2026

@bednar Any chance merging it?

Copy link
Contributor

@bednar bednar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

@bednar bednar merged commit e22be8b into influxdata:master Mar 2, 2026
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Point#to_line_protocol fails with FrozenError under RUBYOPT="--enable-frozen-string-literal

2 participants